Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/#127 프로필 및 채널에 이미지 추가 #130

Merged
merged 6 commits into from
Sep 11, 2023
Merged

Conversation

navyjeongs
Copy link
Member

@navyjeongs navyjeongs commented Sep 9, 2023

🤠 개요

💫 설명

  • 기존에 프로필을 불러오는 API에서 응답 형식이 변경되어서 해당 형식을 적용했어요.

  • 채널의 경우 이미지가 있으면 해당 이미지를 보여주고 채널 이름을 숨겨요

  • 만약 채널에 이미지가 없다면 채널 이름의 두 글자만 표시되도록 우선 변경했어요.

  • 채널 이미지와 관련해서 의논할 부분이 생겼어요.

    • 지금은 원형 전체의 75퍼센트가 사진이고 25퍼센트가 게임 종류를 나타내요.

    • 왼쪽부터 1, 2, 3이라고 할 때 1번이 원본 사진이라고 했을 때 원래라면 2번 처럼 원본 사진의 75퍼센트만 보이고 25퍼센트를 가리면서 게임 종류를 나타내야하는데 지금은 3번처럼 전체 사진을 75%에 맞게 크기를 줄여서 사용해요.

    • 이 부분을 어떻게 2번 처럼 표시할 지 모르겠어요.

ddd
  • 지금 아래와 같은 구조로 채널 컴포넌트를 만들었어요.
<ChannelBtn>
   <div>채널 제목</div>
   <div>게임 종류</div>
</ChannelBtn>


const ChannelBtn = styled.button<{ url?: string }>`
  background: linear-gradient(to bottom, #344051 75%, #202b37 25%);
  ${(prop) =>
    prop.url &&
    css`
      background-image: url(${prop.url});
      background-size: 100% 75%;
      background-position: center top; /* 배경 이미지를 가운데 정렬하고 위에서부터 정렬합니다. */
      background-repeat: no-repeat; /* 배경 이미지 반복을 비활성화합니다. */
    `}
  &:hover {
    border-radius: 30%;
  }
`;
  • 원형 모양을 그려야해서 상위 ChannelBtn에 border-radius: 50%로 원형 모양을 그렸고 ChannelBtn에서 배경색을 미리 지정했어요.
    • ChannelBtn이 아니라 하위 div에서 배경색을 지정하려고 했는데 저기 반원과 비슷한 모양으로 border-radius를 주기 힘들더라구요.

📷 스크린샷 (Optional)

445

@navyjeongs navyjeongs added the ✨ Feat 기능 개발 label Sep 9, 2023
@navyjeongs navyjeongs requested a review from pp449 September 9, 2023 16:42
@navyjeongs navyjeongs self-assigned this Sep 9, 2023
@navyjeongs navyjeongs linked an issue Sep 9, 2023 that may be closed by this pull request
2 tasks
Copy link
Member

@pp449 pp449 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

채널 이미지 관련해서 게임 카테고리의 경우는 hover 했을 때 보이는건 어떨까요??

현재 채널 이름을 2글자만 들어간다면 적은것 같고 이미지를 삽입해도 이미지가 잘 안보일거 같아요
아니면 일단 이대로 적용한뒤 추후 결정해도 좋을거 같아요

@navyjeongs
Copy link
Member Author

채널 이미지 관련해서 게임 카테고리의 경우는 hover 했을 때 보이는건 어떨까요??

현재 채널 이름을 2글자만 들어간다면 적은것 같고 이미지를 삽입해도 이미지가 잘 안보일거 같아요 아니면 일단 이대로 적용한뒤 추후 결정해도 좋을거 같아요

좋은거 같아요! 추후에 이슈를 새로 만들어서 해당 작업을 진행할게요

@navyjeongs navyjeongs merged commit 7d10d5d into dev Sep 11, 2023
@navyjeongs navyjeongs deleted the Feat/#127 branch September 11, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feat: 헤더와 채널에 이미지 src 추가
2 participants